Skip to content

Conversation

@kar-rahul-aws
Copy link
Member

@kar-rahul-aws kar-rahul-aws commented May 26, 2025

Issue #, if available:

Description of changes:
This PR updates all download commands (such as curl) to download files outside the repository content.

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@kar-rahul-aws kar-rahul-aws marked this pull request as draft May 26, 2025 07:32
@kar-rahul-aws kar-rahul-aws changed the title Update curl command to download in a non user accessible folder Update download locations to outside repository content May 26, 2025
@aggarg aggarg marked this pull request as ready for review June 2, 2025 10:55
log-dir: demo_run_logs
timeout-seconds: 300
success-line: "Demo completed successfully."
# - name: Run and monitor the MQTT Serializer Demo
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These demos fail in GH Actions , but not reproducible in local environment, We will address them in a later PR.

log-dir: demo_run_logs
timeout-seconds: 300
success-line: "Demo completed successfully."
# - name: Run and monitor the MQTT Subscription Manager Demo
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These demos fail in GH Actions , but not reproducible in local environment, We will address them in a later PR.

@kar-rahul-aws kar-rahul-aws changed the title Update download locations to outside repository content [ DO NOT MERGE ]Update download locations to outside repository content Jun 2, 2025
@aggarg aggarg changed the title [ DO NOT MERGE ]Update download locations to outside repository content Update download locations to outside repository content Jun 5, 2025
@aggarg aggarg merged commit 2323cd0 into aws:main Jun 5, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants